Skip to content

chore(repo): Update CI caching strategy - #3566

Merged
tmilewski merged 5 commits into
mainfrom
tom/update-ci-caching-strategy
Jun 17, 2024
Merged

chore(repo): Update CI caching strategy#3566
tmilewski merged 5 commits into
mainfrom
tom/update-ci-caching-strategy

Conversation

@tmilewski

@tmilewski tmilewski commented Jun 13, 2024

Copy link
Copy Markdown
Member

Description

Flow:

  1. Restore node_modules ONLY if there's an EXACT match based on package-lock.json
  2. [if no exact match]
    1. Install with npm ci ...
    2. Save immediately after

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@tmilewski tmilewski self-assigned this Jun 13, 2024
@changeset-bot

changeset-bot Bot commented Jun 13, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: be5887c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@tmilewski
tmilewski requested review from a team, brkalow and nikosdouvlis June 13, 2024 22:03
@tmilewski
tmilewski marked this pull request as ready for review June 13, 2024 22:04
@LekoArts
LekoArts enabled auto-merge (squash) June 14, 2024 07:20
@LekoArts
LekoArts disabled auto-merge June 14, 2024 08:22
@tmilewski
tmilewski marked this pull request as draft June 17, 2024 22:39
@tmilewski
tmilewski force-pushed the tom/update-ci-caching-strategy branch from 19c277f to f926eda Compare June 17, 2024 22:47
@tmilewski
tmilewski marked this pull request as ready for review June 17, 2024 23:22
@tmilewski
tmilewski merged commit f8c8a7c into main Jun 17, 2024
@tmilewski
tmilewski deleted the tom/update-ci-caching-strategy branch June 17, 2024 23:22
LekoArts added a commit that referenced this pull request Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants